Skip to content

fix: reduce LMP false positives - #158

Merged
alxxjohn merged 1 commit into
mainfrom
fix-fp-2
Aug 25, 2026
Merged

alxxjohn merged 1 commit into
mainfrom
fix-fp-2

Conversation

@alxxjohn

Copy link
Copy Markdown
Contributor

Summary

  • Retune Code Quality heuristics to avoid LMP false positives around Next.js route/action orchestration, validated boundary inputs, discriminated { ok, value/error } result contracts, DTO/API response shapes, and UI-safe defaults.
  • Narrow noisy naming, duplication, TypeScript assertion, environment-branching, error-handling, and god-module detections for intentional adapter, Supabase/query, test-helper, string-parsing, fallback, and central utility patterns.
  • Add regression coverage for the representative LMP false-positive cases, including environment parsing, test/helper assertions, DTO/generic parser handling, and boundary contract/name hardening.

Tests

  • make fmt-check
  • make lint
  • go test ./tests/checks -run 'Test(LMPFalsePositiveHardening|TypeScriptAssertionRules|DefensiveRulesSkip|QualityEnvironmentBranching)'

@alxxjohn
alxxjohn merged commit 747086a into main Aug 25, 2026
16 checks passed
@alxxjohn
alxxjohn deleted the fix-fp-2 branch August 25, 2026 00:26
alxxjohn added a commit that referenced this pull request Aug 25, 2026
🤖 I have created a release *beep* *boop*
---


##
[1.7.1](v1.7.0...v1.7.1)
(2026-08-25)


### Bug Fixes

* gate Rust dead-code Cargo execution
([0440f94](0440f94))
* reduce LMP false positives
([12c2255](12c2255))
* reduce LMP false positives
([#158](#158))
([747086a](747086a))
* support Go 1.23 in Rust dead-code test
([427c6f6](427c6f6))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant